Skip to content

Conversation

ssitar583
Copy link
Contributor

No description provided.

@ssitar583 ssitar583 requested a review from a team as a code owner April 8, 2025 11:11
@CLAassistant
Copy link

CLAassistant commented Apr 8, 2025

CLA assistant check
All committers have signed the CLA.

@rdkcmf-jenkins
Copy link
Contributor

b'## Blackduck scan failure details

Summary: 0 violations, 0 files pending approval, 1 file pending identification.

  • Protex Server Path: /home/blackduck/github/entservices-inputoutput/45/rdkcentral/rdkservices

  • Commit: 652aba8

Report detail: gist'

@ssitar583 ssitar583 force-pushed the topic/RDKEMW-1014 branch from 652aba8 to b7fa382 Compare April 8, 2025 11:39
@rdkcmf-jenkins
Copy link
Contributor

b'## Blackduck scan failure details

Summary: 0 violations, 0 files pending approval, 1 file pending identification.

  • Protex Server Path: /home/blackduck/github/entservices-inputoutput/45/rdkcentral/rdkservices

  • Commit: b7fa382

Report detail: gist'

@rdkcmf-jenkins
Copy link
Contributor

b'## Blackduck scan failure details

Summary: 0 violations, 0 files pending approval, 1 file pending identification.

  • Protex Server Path: /home/blackduck/github/entservices-inputoutput/45/rdkcentral/rdkservices

  • Commit: b7fa382

Report detail: gist'

@rdkcmf-jenkins
Copy link
Contributor

b'## WARNING: A Blackduck scan failure has been waived

A prior failure has been upvoted

  • Upvote reason: OK

  • Commit: b7fa382
    '

static HdcpProfile* _instance;
namespace Plugin {

class HdcpProfile : public PluginHost::IPlugin, public PluginHost::JSONRPC
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A minor comment: The indentation doesn't look correct. Please fix this.

_parent.Deactivated(connection);
}

void OnDisplayConnectionChanged(const Exchange::IHdcpProfile::HDCPStatus hdcpstatus) override
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you change this parameter "hdcpstatus" to call by referece "&hdcpstatus"?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants